Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Swift version #6612

Closed
wants to merge 4 commits into from
Closed

Update Swift version #6612

wants to merge 4 commits into from

Conversation

rcombs
Copy link
Contributor

@rcombs rcombs commented Apr 5, 2019

Latest Xcode (10.2) drops Swift 3 support, so this needed doing.
It also drops the static Swift libs in favor of dynamic; on current macOS (10.14.4), those libs are found at /usr/lib/swift, so this PR uses that, but I'm not sure if that's true on older versions; we might have to add something to ship them in redistributable builds somehow.

Also fixes some deprecation warnings in Obj-C code, and silences all OpenGL deprecation warnings, since they're pretty noisy.

@Akemi
Copy link
Member

Akemi commented Jul 22, 2019

macOS: silence OpenGL deprecation warnings seems fine as it is, i would just prefix it with mac: for some consistency.

Cocoa: fix deprecation warnings is not mergeable since it will break the d'n'd the same way as the those swift changes. it needs more adjustments. see what i had to do on cocoa-cb.

msk pushed a commit to petabi/pkgsrc that referenced this pull request Aug 30, 2019
According upstream pull request 6612
(<mpv-player/mpv#6612>) and as noticed
via PR pkg/54426 newer Xcode (10.2) dropped support for Swift 3
that is used in mpv-0.29.1.

Workaround that by disabling swift support.

This will probably not needed and should be removed in next stable
mpv version.
@jeeb
Copy link
Member

jeeb commented Sep 28, 2019

  1. The Swift stuff has already been fixed in master by @Akemi
  2. I have now merged the OpenGL deprecation silencing for C/objC as c940d7d .
  3. The pasteboard stuff needs reworking, which really can go into a new PR.

Thus, closing this. Thanks to everyone.

@jeeb jeeb closed this Sep 28, 2019
@Akemi Akemi mentioned this pull request Feb 26, 2020
@rcombs rcombs deleted the swift-update branch June 7, 2022 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants